Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow admin password to be set by config if present. #4429

Merged
merged 3 commits into from
Feb 28, 2022
Merged

Allow admin password to be set by config if present. #4429

merged 3 commits into from
Feb 28, 2022

Conversation

pavlosdan
Copy link
Contributor

Motivation
Currently BLT always generates a random password for user 1. On local environments we want to be able to define a username and password so we don't have to reset it after every single setup or use drush uli to get into the site.

Proposed changes
Since site:install allows for the --account-pass option, if a password is given in the blt config files, use it.

Testing steps

  1. apply this patch
  2. without changing anything else, run blt setup:drupal:instal -y --no-interaction
  3. confirm random user name and password
  4. add drupal.account.pass to your blt.yml file (optionally define a drupal.account.name as well for an easier test).
  5. re-run blt setup:drupal:instal -y --no-interaction
  6. confirm admin name and password are set to your custom ones defined in step 4.

@danepowell
Copy link
Contributor

Thanks for the PR, looks good. I added the new drupal.account.pass config to build.yml.

@danepowell danepowell added Enhancement A feature or feature request Minor change labels Feb 28, 2022
@danepowell danepowell merged commit e7de9f9 into acquia:main Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request Minor change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants